Skip to content

Pin task version to 3.9.0 because of the PATH on win bug #1576

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Nov 30, 2021

Conversation

umbynos
Copy link
Contributor

@umbynos umbynos commented Nov 30, 2021

Please check if the PR fulfills these requirements

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • The PR follows
    our contributing guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • UPGRADING.md has been updated with a migration guide (for breaking changes)
  • What kind of change does this PR introduce?

Infrastructure fix

  • What is the current behavior?

The Test Go workflow on Windows is broken after the newest release of Task (3.9.1) and was working fine with 3.9.0
The problem should be linked to the upgrade in mvdan/sh they did in 3.9.1.
The bug is present only on Windows, apparently the PATH env variable is not expanded correctly under subcommands. Log attached for further details

  • What is the new behavior?

To mitigate the problem, we decided to pin the task version used to 3.9.0 in the workflows

no

  • Other information:

logs_46403.zip


See how to contribute

@umbynos umbynos added os: windows Specific to Windows operating system topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project labels Nov 30, 2021
@umbynos umbynos requested a review from per1234 November 30, 2021 14:59
@umbynos umbynos self-assigned this Nov 30, 2021
@umbynos umbynos changed the title pin task version to verify if can recognize correctly the PATH on win Pin task version to 3.9.0 because of the PATH on win bug Nov 30, 2021
@umbynos umbynos merged commit 55ad9ec into master Nov 30, 2021
@umbynos umbynos deleted the umbynos/pin_task branch November 30, 2021 16:34
umbynos added a commit that referenced this pull request Dec 3, 2021
umbynos added a commit that referenced this pull request Dec 3, 2021
umbynos added a commit that referenced this pull request Dec 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
os: windows Specific to Windows operating system topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants